Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enable opencl on ffmpeg5-6 with DSM >= 7 #6166

Merged
merged 88 commits into from
Sep 22, 2024
Merged

Enable opencl on ffmpeg5-6 with DSM >= 7 #6166

merged 88 commits into from
Sep 22, 2024

Conversation

th0ma7
Copy link
Contributor

@th0ma7 th0ma7 commented Jul 3, 2024

Description

This PR superseeds #6158 by downgrading the build to force using LLVM 14.x mandatory needed to build igc (intel-graphic-compiler).

Fixes #6017

Checklist

  • Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Bug fix
  • New Package
  • Package update
  • Includes small framework changes
  • This change requires a documentation update (e.g. Wiki)

@th0ma7 th0ma7 self-assigned this Jul 3, 2024
This was referenced Jul 4, 2024
Closed
Closed
Closed
This was referenced Jul 12, 2024
Open
Closed
This was referenced Aug 20, 2024
Merged
Closed
Closed
This was referenced Aug 29, 2024
Merged
Merged
@th0ma7
Copy link
Contributor Author

th0ma7 commented Sep 15, 2024 •
edited
Loading

@hgy59 I just noticed a not enough space... Looking at my own build status locally enabling opencl is space hungry:

spksrc@spksrc:~/ffmpeg-opencl-140/spksrc/spk/synocli-videodriver$ du -sh work-x64-*
3.3G	work-x64-6.2.4
18G	work-x64-7.1

I'll look at cleaning-up post-build what is really resource intensive... Wonder if there is some easy way to augment that? and wetter @Safihre we do have some $ that could help?

EDIT: Adding for future reference:

------>> work-x64-6.2.4 <<-------
11848	libva-utils-2.17.1
18312	gmmlib-intel-gmmlib-22.3.4
29976	libvpl-2.13.0
72128	intel-vaapi-driver-2.4.1
662312	MediaSDK-intel-mediasdk-22.5.4
2204672	media-driver-intel-media-22.5.4
Install directory:
398016	install

------>> work-x64-7.1 <<-------
12348	libva-utils-2.22.0
14868	SPIRV-LLVM-Translator-14.0.3
18520	gmmlib-intel-gmmlib-22.4.1
20108	libvpl-tools-1.2.0
25440	level-zero-1.17.28
30064	libvpl-2.13.0
60708	SPIRV-Tools-2024.2
72788	intel-vaapi-driver-2.4.1
96280	intel-graphics-compiler-igc-1.0.17384.12
520328	compute-runtime-24.31.30508.7
581920	MediaSDK-intel-mediasdk-23.2.2
1375804	llvm-project-14.0.5.src
2538568	media-driver-intel-media-24.2.5
10238496	igc.build
Install directory:
2330520	install

@Safihre
Copy link
Contributor

Safihre commented Sep 15, 2024

I do, I offered a couple of times. Just need a PayPal to donate it to!

@th0ma7
Copy link
Contributor Author

th0ma7 commented Sep 15, 2024

I do, I offered a couple of times. Just need a PayPal to donate it to!

Good to know, I opened-up a ticket with github to check if there isn't any possibility to be granted with more disk storage, otherwise it becomes paid to which I'll have to figure out how this works. Note that the $ would also be beneficial to increase our server storage space... but that's another issue all together. I'll keep you posted when I hear back.

@th0ma7 th0ma7 mentioned this pull request Sep 15, 2024
Merged
10 tasks
@th0ma7
ffmpeg56: Add SPK_DEPENDS and adjust changelog
b72b542
@th0ma7
vc-intrinsics: Update from version 0.19.0 to 0.20.0
89e3e4a
@th0ma7
intel-opencl-clang-140: Align with IGC for static build
f7245b9 
Also remove vc-intrinsics build fix as issue now solved with
new version 0.20.0 with ref. PR
intel/vc-intrinsics#130
@th0ma7
intel-level-zero: Update from 1.17.28 to 1.17.42
8afdcd0
@th0ma7
intel-gmmlib-latest: Update from 22.4.1 to 22.5.2
74c648a
@th0ma7
intel-graphics-compiler: Update from 1.0.17384.12 to 1.0.17537.20
b7fb5c9
@th0ma7
cmake: Disable developer warnings (equivalent of -Wno-dev)
5b07bb0
@th0ma7
extract.mk: Allow extra parameter to tar extract commands
a464bcd
@th0ma7
Add a new GCC_NO_DEBUG_INFO disabling symbols at build-time
229f447 
This remove debug symbols at build time reducing binary size
significantly + reducing total compile+link time.

Also, when set this enforces a "clean" of the current source
directory (i.e. build for cmake/meson/ninja) to drop post-build
total storage space used.

It is hoped that it will allow compiling intel-graphics-compiler
(IGC) using github-action which currently uses way to much space.
@th0ma7
intel-compute-runtime: Do not enable GCC_NO_DEBUG_INFO has failing
20858ff
@th0ma7
intel-mediasdk: Not enable GCC_NO_DEBUG_INFO for ffmpeg --enable-mfx
f1c59e4
@th0ma7
ffmpeg.mk: Add new ffmpeg build functionality
4b4dbeb
@th0ma7
videodriver.mk: Fix
6b5cea5
@th0ma7
tvheadend: Update to new version & migrate to new ffmpeg.mk
d7c1d85
@th0ma7
ffmpeg56: Update changelog on to using new synocli-videodriver
c59498c
@th0ma7
comskip: Migrate to using new ffmpeg.mk
4935c71
@th0ma7
chromaprint: Migrate to ffmpeg6
e69067e
@th0ma7 th0ma7 changed the title [WIP] Enable opencl on >= ffmpeg6 with DSM >= 7 [WIP] Enable opencl on ffmpeg5-6 with DSM >= 7 Sep 18, 2024
@th0ma7 th0ma7 linked an issue Sep 19, 2024 that may be closed by this pull request
Lastest ffmpeg6 not work with QSV #6174
Closed
1 task
@th0ma7 th0ma7 merged commit 0a4d096 into SynoCommunity:master Sep 22, 2024
17 checks passed
@mreid-tt mreid-tt changed the title [WIP] Enable opencl on ffmpeg5-6 with DSM >= 7 Enable opencl on ffmpeg5-6 with DSM >= 7 Sep 22, 2024
@CamFlyerCH
Copy link

Thanks to make Jellyfin usable again.

@th0ma7
Copy link
Contributor Author

th0ma7 commented Sep 24, 2024

@CamFlyerCH thnx for testing it out. Updated packages now published online.

@th0ma7 th0ma7 deleted the ffmpeg-opencl-140 branch September 24, 2024 21:08
@th0ma7 th0ma7 added the status/published Published and activated (may take up to 48h until visible in DSM package manager) label Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@th0ma7 th0ma7

Labels
status/published Published and activated (may take up to 48h until visible in DSM package manager)
Projects
None yet
Milestone
No milestone
3 participants
@th0ma7 @Safihre @CamFlyerCH